Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bgpd: make as-path-loop-detection conform to the framework #15553

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

fdumontet6WIND
Copy link
Contributor

currently:
when as-path-loop-detection is set on a peer-group.
members of the peer-group are not using that functionnality.

analysis:
the as-path-loop-detection, is not using the peer's flags
related framework.

fix:
use the peer's flag framework for as-path-loop-detection.

Signed-off-by: Francois Dumontet [email protected]

@frrbot frrbot bot added bgp tests Topotests, make check, etc labels Mar 14, 2024
@fdumontet6WIND fdumontet6WIND force-pushed the as_path_loop branch 2 times, most recently from 266f048 to e82475c Compare March 14, 2024 11:26
currently:
when as-path-loop-detection is set on a peer-group.
members of the peer-group are not using that functionnality.

analysis:
the as-path-loop-detection, is not using the peer's flags
related framework.

fix:
use the peer's flag framework for as-path-loop-detection.

Signed-off-by: Francois Dumontet <[email protected]>
@fdumontet6WIND fdumontet6WIND force-pushed the as_path_loop branch 3 times, most recently from d1b1431 to a294cc3 Compare March 14, 2024 16:29
add a tests of setting as-path-loop-detection through peer-group

Signed-off-by: Francois Dumontet <[email protected]>
Copy link
Member

@ton31337 ton31337 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fdumontet6WIND
Copy link
Contributor Author

ci:rerun

@fdumontet6WIND
Copy link
Contributor Author

failure not related to bgp
ci:rerun

@ton31337 ton31337 merged commit 0ecea59 into FRRouting:master Mar 18, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bgp master size/L tests Topotests, make check, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants